-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(WIP) Prototyping the new Accounts DB storage #28790
Conversation
Hello, @jeffwashington. If you want to test out the PR, this can be done by compiling the ledger-tool and run the following command:
Here's my script for running it for example:
The above command will create the new accounts-data-storage file from the specified append_vec. It will also verify whether the two files have the same account meta and data. For each account, the command will dump its data from both append_vec and the new format and verify whether they are the same:
In my experiment, the new format could save up to 75% of the storage size:
|
@@ -4330,6 +4350,78 @@ fn main() { | |||
eprintln!("{}", err); | |||
} | |||
} | |||
("new_ads_file", Some(arg_matches)) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeffwashington: FYI: this is the command that creates the new accounts-data-storage from from an append_vec file and verify its correctness.
WIP.
Based on #28550